Skip to content

Comments

BigNumber integration#21

Open
AndriianChestnykh wants to merge 33 commits intotetherto:developfrom
idyllicvision:feature/bignumber-integration
Open

BigNumber integration#21
AndriianChestnykh wants to merge 33 commits intotetherto:developfrom
idyllicvision:feature/bignumber-integration

Conversation

@AndriianChestnykh
Copy link

@AndriianChestnykh AndriianChestnykh commented Dec 19, 2025

Description

Replace JS native number with BigNumber for token prices, amounts and fees

Motivation and Context

JS native number can have rounding errors at arithmetic operations, which may be an issue for precision critical operations like token/crypto prices, balances, amounts and fees

sergiy-zvyahelskiy and others added 30 commits December 1, 2025 19:17
refactor: replace custom mnemonic validation with bip39.validateMnemonic
Add address validation for existing networks
- Added utils/bignumber.ts
- Implemented formatAmountBN and formatTokenAmountBN
- Updated AssetsScreen to use BigNumber
- Added getFiatValueBN method to PricingService
…port

- Disabled `import/no-named-as-default` to avoid false positives with BigNumber
- Added `"baseUrl": "."` to tsconfig for proper alias resolution
- Updated import style to `import BigNumber from 'bignumber.js'` for better readability
@AndriianChestnykh AndriianChestnykh changed the title Feature/bignumber integration BigNumber integration Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants